Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PDOモジュール入れてるけどSQLite3クラスを使ってたのでPDOクラスを利用するように変更する #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rururu3
Copy link
Contributor

@rururu3 rururu3 commented Nov 19, 2016

それにともなって設定ファイルにpdoのdns設定できるようにする(これによりDBをMySQLなどにすることも可能・・・ですが、セッション2つ貼る意味を理解してないのでオリジナル通り2つのまま)
ついでにテーブルにインデックスを貼る

Azureで環境作って
curl http://localhost/aaaa
で5回したら弾かれるのまではテストしました。

それにともなって設定ファイルにpdoのdns設定できるようにする(これによりDBをMySQLなどにすることも可能・・・ですが、セッション2つ貼る意味を理解してないのでオリジナル通り2つのまま)
ついでにテーブルにインデックスを貼る
@disco-v8
Copy link
Owner

たしかPDOクラスを入れないとSQLite3関連が使えなかったので、インストールするようにしています。
で、まぁぶっちゃけPDO経由でいろんなSQLに対応してもよかったんだけど、fail2banと同じ環境で動くことをゴールとしたので、SQLite3を使っています。(まぁわかる人はみればPDO入っているしMySQLなりPostgreSQL使うかなぁと)

セッションを二つ貼っているのは、一つはカウンタDBに対して、もう一つはBANしているDBに対して、で、二つのDBにしたのは、いろいろ考えてDBを分けたかったから、です。(SQLite3はDB毎にファイルができるから)

というわけでどうでしょ?

@rururu3
Copy link
Contributor Author

rururu3 commented Nov 30, 2016

セッションわけに関して了解です。

PDO入っているしMySQLなりPostgreSQL使うかなぁと)
という部分はこのリクエストで対応してるのですがいかがでしょ?

@disco-v8
Copy link
Owner

ban4ipd_init.phpのパッチを見たけど、これだとdb_count_pdo_dsnの有無による処理がif{}else{}と、try{}catch{}となっていてわかりずらいのと、そもそもこの変数の宣言がない時に無条件でSQLite3でDBを新規に作ってしまっている?ように読めるんだけど、それはどうなんだろう?(そこをエラーにしないのはなんで?)
という感じですー

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants